Skip to content

ZA2 Patou Mabando / HTML-CSS-Module-Project#367

Open
PatouMabando wants to merge 4 commits into
CodeYourFuture:masterfrom
PatouMabando:master
Open

ZA2 Patou Mabando / HTML-CSS-Module-Project#367
PatouMabando wants to merge 4 commits into
CodeYourFuture:masterfrom
PatouMabando:master

Conversation

@PatouMabando
Copy link
Copy Markdown

I added HTML and CSS to this file using flexbox CSS

Volunteers: Are you marking this coursework? You can find a guide on how to mark this coursework in HOW_TO_MARK.md in the root of this repository

Your Details

  • Your Name: Patou Malongi Mabando
  • Your City: ZA2
  • Your Slack Name: Patou Mabando

Homework Details HTML-CSS-Module-Project

  • Module: HTML CSS Layout • Flexbox • Grid
  • Week: 3

Notes

  • What did you find easy? The Layout

  • What did you find hard?

  • What do you still not understand?

  • Any other notes?

I added HTML and CSS to this file using flexbox CSS
@PatouMabando PatouMabando changed the title ZA2 Patou Mabando I added HTML and CSS ZA2 Patou Mabando Jun 16, 2022
@PatouMabando PatouMabando changed the title ZA2 Patou Mabando ZA2 Patou Mabando / HTML-CSS-Module-Project Jun 16, 2022
Comment thread index.html Outdated
Comment thread index.html
<!-- Remember: Use semantic HTML tags like <header>, <main>, <nav>, <footer>, <section> etc -->
<!-- All the images you need are in the 'img' folder -->
<header>
<div class="header container">
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This div with the class header container is not looking good, however, your header must a least have 2 child elements, the logo, and the nav list, organizing it this way will make your CSS much clear.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can You please suggest to me how I can write it down please. Thanks

Comment thread index.html
<figcaption>Pay as You Go</figcaption>
</figure>

</ul>
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Writing your ul inside your figure tag would be much better than keeping it outside the figure, please re-organize your HTML structure in this area,

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can You please suggest to me how I can write it down please. Thanks

Copy link
Copy Markdown

@Mathias02 Mathias02 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Patou, would you please review my Karma project and suggest changes in the pull request

All the the wrong tag in my HTML on top are removed. 
Thank you Mathias, I did pay attention on that mistake.
Copy link
Copy Markdown
Author

@PatouMabando PatouMabando left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank Mathias.
I removed all the wrong tag in HTML file.
For your last two suggestion, can you please suggest to me how I can write it down please. Thanks

Comment thread index.html Outdated
Comment thread index.html
<!-- Remember: Use semantic HTML tags like <header>, <main>, <nav>, <footer>, <section> etc -->
<!-- All the images you need are in the 'img' folder -->
<header>
<div class="header container">
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can You please suggest to me how I can write it down please. Thanks

Comment thread index.html
<figcaption>Pay as You Go</figcaption>
</figure>

</ul>
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can You please suggest to me how I can write it down please. Thanks

I update the images link.
I add the background image link in Hero-container CSS
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants